New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
move internal apps API types out of extensions package #70028
move internal apps API types out of extensions package #70028
Conversation
b63cd06
to
e84f7e8
Compare
e84f7e8
to
5b629bd
Compare
5b629bd
to
ea8dd57
Compare
/assign @kow3ns |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
/assign @caesarxuchao |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Fix the nits if you have to rebase anyway.
@@ -101,7 +101,7 @@ func TestSetControllerConversion(t *testing.T) { | |||
}, | |||
} | |||
|
|||
t.Logf("rs._internal.extensions -> rs.v1beta1.extensions") | |||
t.Logf("rs._internal.apps -> rs.v1beta1.apps") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: s/v1beta1.apps/v1.apps
@@ -116,7 +116,7 @@ func TestSetControllerConversion(t *testing.T) { | |||
), | |||
) | |||
|
|||
t.Logf("rs.v1beta1.extensions -> rc._internal") | |||
t.Logf("rs.v1beta1.apps -> rc._internal") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same nit
47ece9b
to
572dfe6
Compare
New changes are detected. LGTM label has been removed. |
nits fixed, retagging need /approve from @caesarxuchao for replication controller package and @timothysc for kubeadm package |
/assign @timothysc transitive dependency changed from extensions to apps, updated cmd/kubeadm/.import-restrictions |
/approve from the kubeadm side. |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: caesarxuchao, liggitt, soltysh, timothysc The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest Review the full test history for this PR. Silence the bot with an |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
related to #43214
Special notes for your reviewer:
The main commits that need review:
Change registration and custom conversion from extensions to apps
fixup extensions->apps references
The "move ..." commits are pure moves, no internal changes.
Does this PR introduce a user-facing change?: